Skip to content

feat: add CI workflow and README badges - #17

Merged
dsiddharth2 merged 2 commits into
mainfrom
feat/readme-badges
May 8, 2026
Merged

feat: add CI workflow and README badges#17
dsiddharth2 merged 2 commits into
mainfrom
feat/readme-badges

Conversation

@dsiddharth2

Copy link
Copy Markdown
Owner

@

Summary

  • Add GitHub Actions CI workflow (ci.yml) that runs pytest on Python 3.11/3.12 and verifies the Docker build on every push/PR to main
  • Add README badges: CI status, Docker publish status, Docker pulls, image size, MIT license, Python version

Test plan

  • Verify CI workflow runs on this PR
  • Confirm badges render correctly on the README
  • Check Docker build step passes in CI
    @

Add GitHub Actions CI pipeline (pytest on 3.11/3.12 + Docker build
verification) and README badges for CI status, Docker pulls, image
size, license, and Python version.
- Add required summary field to test fixtures (conftest, phase2_scoring,
  post_findings) after schema made it mandatory
- Fix MagicMock leaking into ADO SDK in test_review_job by removing
  wholesale models.review_models mock
- Update stale prompt assertion in test_turn_budget to match refactored
  pre-injected context prompt
- Fix review_modes fixture key (singular → plural) in test_batch_review
- Fix graph_builder test to check caplog instead of capsys
- Add summary=None to MagicMock FindingsFile in summary markdown tests
@dsiddharth2
dsiddharth2 force-pushed the feat/readme-badges branch from 3fee113 to 805fc57 Compare May 8, 2026 06:54
@dsiddharth2
dsiddharth2 merged commit b43822e into main May 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant